Skip to content

Fix links#11554

Merged
gui1117 merged 4 commits intomasterfrom
gui-fix-links
Apr 5, 2026
Merged

Fix links#11554
gui1117 merged 4 commits intomasterfrom
gui-fix-links

Conversation

@gui1117
Copy link
Copy Markdown
Contributor

@gui1117 gui1117 commented Mar 30, 2026

No description provided.

@gui1117 gui1117 requested review from a team and cheme as code owners March 30, 2026 06:18
#[command(about = "Distributed statement store latency benchmark", long_about = None)]
struct Args {
/// Comma-separated list of RPC WebSocket endpoints (e.g., ws://node1:9944,ws://node2:9944)
/// Comma-separated list of RPC WebSocket endpoints (e.g., `ws://node1:9944,ws://node2:9944`)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was interpreted as file ws:/node... by lychee.

Comment thread .config/lychee.toml
"%7B%7D",
"%7B:\\?}",
# Example ws:// URLs in Rust doc comments incorrectly parsed as file paths by lychee
"file://.*/ws:/node.*",
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

according to claude, lychee is broken for //! doc and it can't parse and exclude the ```bash part

@gui1117 gui1117 added R0-no-crate-publish-required The change does not require any crates to be re-published. T11-documentation This PR/Issue is related to documentation. labels Mar 30, 2026
@gui1117 gui1117 enabled auto-merge April 5, 2026 00:12
@gui1117 gui1117 added this pull request to the merge queue Apr 5, 2026
Merged via the queue into master with commit 996adb4 Apr 5, 2026
328 of 346 checks passed
@gui1117 gui1117 deleted the gui-fix-links branch April 5, 2026 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

R0-no-crate-publish-required The change does not require any crates to be re-published. T11-documentation This PR/Issue is related to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants